Skip to content

ci: add AI PR review workflow - #477

Merged
margaretjgu merged 3 commits into
mainfrom
ci/ai-pr-review
Jul 22, 2026
Merged

ci: add AI PR review workflow#477
margaretjgu merged 3 commits into
mainfrom
ci/ai-pr-review

Conversation

@margaretjgu

Copy link
Copy Markdown
Member

Summary

Adds an AI PR review workflow using the existing LITELLM_API_KEY secret (already provisioned for auto-pr.yml).

What it does:

  • Runs on every non-draft PR open/update
  • Sends the diff (capped at 32KB) to Claude via the LiteLLM gateway
  • Posts a review comment focused on real bugs, missing error handling, logic errors, security issues
  • Deletes stale bot comments on re-runs so the thread stays clean

What it skips: style nits, summaries of what the code does, anything not actionable.

No new secrets or provisioning needed - reuses LITELLM_API_KEY already in the repo.

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 1 0 0 0.25s
✅ COPYPASTE jscpd yes no no 0.97s
✅ REPOSITORY gitleaks yes no no 58.43s
✅ REPOSITORY git_diff yes no no 0.72s
✅ REPOSITORY secretlint yes no no 30.7s
✅ REPOSITORY trivy yes no no 16.54s
✅ YAML yamllint 1 0 0 0.45s

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

See detailed reports in MegaLinter artifacts
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@margaretjgu
margaretjgu force-pushed the ci/ai-pr-review branch 2 times, most recently from 26c8eef to a496b49 Compare July 14, 2026 21:39
@margaretjgu
margaretjgu merged commit a4c2c06 into main Jul 22, 2026
29 checks passed
@margaretjgu
margaretjgu deleted the ci/ai-pr-review branch July 22, 2026 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants